Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TASK-49] style: 디자인 시스템 변경사항 반영 (국문/영문 폰트 설정, 공용 컴포넌트 기능 추가) #12

Merged
merged 16 commits into from
Dec 24, 2024

Conversation

dahyeo-n
Copy link
Member

📝 작업 내용

  1. Pretendard 폰트를 기본으로 지정, 영문에 적용될 Montserrat 폰트는 classNamefont-sans CSS 클래스를 작성하면 적용되도록 추가
  2. 아이콘 추가 (Dropup/Dropdown, Lock/Unlock)
  3. FilterDropdown 컴포넌트
    3-1) 아이콘 변경 (Dropup/Dropdown)
    3-2) 선택된 옵션 강조 기능 추가
  4. Pagination 컴포넌트
    4-1) MoreHorizontal 아이콘 추가
    4-2) 마지막 페이지 표시
    4-3) 페이지 이동 시 버튼 위치 고정 및 페이지 그룹 전환 로직 구현

📸 스크린샷 (선택)

image

💬 전달사항

  1. Button 컴포넌트는 상우님이 구현하시는 <메인 페이지>에선 사용되지 않고, 제가 구현하는 <작품 목폭 페이지>에서만 사용되니 예림님과 좀 더 의논한 뒤 디자인 확정되면 그때 수정하겠습니다 :)
  2. 유효성 검사 변수명 통일 반영했습니다 (check → validate)

@dahyeo-n dahyeo-n added chore 그 외 기타 수정 (ex. 오타 등) refactor 코드 리팩토링, 주석 삭제 feat 새로운 기능 추가 style 사용자 UI 디자인 변경 Priority: High 우선순위 높음 labels Dec 24, 2024
@dahyeo-n dahyeo-n requested a review from SangWoo9734 December 24, 2024 12:58
Copy link
Collaborator

@SangWoo9734 SangWoo9734 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

늦은시간에 리뷰드려 죄송합니다!

전체적인 부분은 크게 수정할 부분이 없어보입니다! 코멘트 확인부탁드립니다!

@@ -22,7 +22,7 @@ const EmailInput = ({ mode }: EmailInputProps) => {
setEmail('');
};

const checkEmailStatus = (email: string) => {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

혹시 이 부분은 제가 수정해도 괜찮을까요? validate 쪽이 많이 바뀔 것 같아서 제가 한 번에 수정하겠습니다!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아항 네 좋습니다 :)

<div className='rounded-full bg-white w-8 h-8 flex-shrink-0'></div>
<button
className='button-m bg-main px-6 py-2 rounded-full text-white flex-shrink-0'
style={{ whiteSpace: 'nowrap' }}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

여기만 인라인 스타일 적용하신 이유가 있을까요? whitespace-nowrap으로는 적용이 안되나요..?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Layout 스타일 적용이 안 되는 부분이 있어서 이것저것 style 로 시도하다가 미처 삭제하는 걸 잊었네요 ㅠㅠ whitespace-nowrap 없어도 디자인한 UI대로 나와서 제거해서 push 했습니다!

@dahyeo-n dahyeo-n merged commit 8da7cff into dev Dec 24, 2024
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore 그 외 기타 수정 (ex. 오타 등) feat 새로운 기능 추가 Priority: High 우선순위 높음 refactor 코드 리팩토링, 주석 삭제 style 사용자 UI 디자인 변경
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants